Skip to content

Conversation

@Prashant-Microsoft
Copy link
Contributor

@Prashant-Microsoft Prashant-Microsoft commented Feb 25, 2025

Purpose

This pull request includes several changes to update the configuration for Application Insights from using the instrumentation key to using the connection string. The most important changes include updates to environment variables, configuration files, and application code.

Updates to environment variables and configuration files:

  • README.md: Changed the environment variable from APPLICATIONINSIGHTS_INSTRUMENTATION_KEY to APPLICATIONINSIGHTS_CONNECTION_STRING.
  • deploy/macae-continer-oc.json: Updated the parameter name from APPLICATIONINSIGHTS_INSTRUMENTATION_KEY to APPLICATIONINSIGHTS_CONNECTION_STRING.
  • deploy/macae-continer.bicep: Changed the configuration to use APPLICATIONINSIGHTS_CONNECTION_STRING instead of APPLICATIONINSIGHTS_INSTRUMENTATION_KEY.
  • src/backend/.env.sample: Replaced APPLICATIONINSIGHTS_INSTRUMENTATION_KEY with APPLICATIONINSIGHTS_CONNECTION_STRING.

Updates to application code:

  • src/backend/app.py: Modified the code to retrieve APPLICATIONINSIGHTS_CONNECTION_STRING instead of APPLICATIONINSIGHTS_INSTRUMENTATION_KEY from environment variables.
  • src/backend/event_utils.py: Updated the function to use APPLICATIONINSIGHTS_CONNECTION_STRING for tracking events.

Updates to tests:

Does this introduce a breaking change?

  • Yes
  • No

@Roopan-Microsoft Roopan-Microsoft merged commit 3f5728d into hotfix Feb 25, 2025
7 checks passed
@github-actions
Copy link

🎉 This PR is included in version 1.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Prashant-Microsoft Prashant-Microsoft deleted the PSL-BUG-14644 branch April 3, 2025 07:10
blessing-sanusi pushed a commit that referenced this pull request Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants